Merge pull request #163 from tormet/master

Torsten Metzner ads support for FIT version 2.0.

Added support for FIT version 2.0
With FIT version 2.0 developer fields where added to the definition
messages. These fields extends the definiton messages and due to this the
number of records which must be parsed afterwards.

FIT files containing developer fields could not be parsed before
this changes.

Simple support was added by just parsing the data but without using them.
It is not expected that any of the currently used data like the
position or the elevation is coded inside the developer fields
because for this we have the already existing records from
FIT version 1.0

Added a test with a fit test file coming from the Wahoo Element Bolt
which uses developer fields from FIT version 2.0

Merge pull request #162 from cweiske/patch-2

Document that Qt5 is required in Build chapter.

Qt5 is required

Qt4 cannot be used anymore - see https://sourceforge.net/p/gpsbabel/mailman/message/36264877/

Fix typo in geojson docs (#161)

fix bugs specific to big endian machines (#155)

* round up some loose endians in mapbar.

* round up another endian, this time in mapsend.

* chase the endians out of naviguide.
also, a potential bug with Qstring length vs. encoded string length.

fixes for the position and track filters. (#157)

the position filter could corrupt the route_waypt_ct for routes/tracks.
the track filter underestimated speed when there were multiple
waypoints with the same time.

garmin gpi reader and writer enhancements (#154)

* enhance garmin gpi reader and writer.
1. fix a few memory leaks.
2. properly encode category passed from user to writer.
3. add support for reading gpi files with dual language strings.
4. add support for code page 65001, i.e. utf8.
5. make sure writer embeds encoded string lengths instead of the lengths of source QStrings.
6. enhance test to cover all of the above.

* correct reference mode.

* fix garmin_gpi test for windows.

* rename garmin_gpi country code to language code.

Prefer QLatin1String for conversion from char strings (#152)

* Prefer QLatin1String for conversion from char strings
as opposed to QStringLiteral or QString when used with
QString::compare or QStringRef::compare.

generalize make_windows_release for MinGW or MSVC. (#151)

make_windows_release.ps1 now can be used with MinGW or MSVC.  It is likely we will have to switch compilers with Qt  >= 5.6 because MinGW is not supported with webengine, and Qt >= 5.6 release packages from www.qt.io distributes webengine and not webkit (although some other parties build webkit from source with these versions of Qt).
appveyor.yml is modified to run a release on one MinGW case (Qt 5.5, webkit) and one MSVC case (Qt 5.6, webengine, 32 bit).  In the case of MSVC the VC redistributible installer is run by setup.
Note that inno setup compiler is always run in 32 bit mode, so if you use this with 64 bit executables they will install in the wrong place, i.e. "Program Files (x86)"

allow passing of multiple possible extensions to GUI (#149)

The extensions are used by GUI for filtering files to display for both input and output.  The first extension is used by the gui in the output file dialog if a user doesn't supply and extension.

add support for shape files with non us-ascii filenames. (#148)

Merge pull request #145 from tsteven4/webengine

Webengine

Merge branch 'master' into webengine

modify igc writer for consistent rounding. (#146)

gui map js beautification and corrections.

delete obsolete & unused html files.

update gmapbase.html for modern stds compliance.

improve js debug & cleanup unused map stuff.

add debug clicker logging back, restore page name.

Merge branch 'webengine' of https://github.com/tsteven4/gpsbabel into webengine

update to google maps api v3.

Merge pull request #143 from gpsbabel/fix_bad_merge

Fix a bad merge base in PR 126. Only main.cc seems to be affected.

Fix includes after repairing 126.

Fix bad git merge in main.cc from PR 126

Merge pull request #126 from gpsbabel/cleanup

Header sorting, moving to C++11. Iterator simplifications.

make windows release script work for any clone name. (#140)

Extend KML, so that <TimeSpan> can be used to describe the time span (#115)

Simplify time in gopal.

Merge branch 'master' into cleanup

Merge remote-tracking branch 'origin/master' into webengine

add utf8 file name test to windows appveyor builds. (#138)

Better handling of UTF-8 paths (#87)

* Better handling of UTF-8 paths

Remove gpsbabel calls to qPrintable which destroys non-ANSI path names on Windows. Replace with QString::toUtf8() and modify low-level file handling code to use _wfopen() and other wide-char functions on Windows to support non-ASCII paths. (I did an experiment to convince myself that fopen() doesn't do UTF-8, and that _wfopen() correctly creates a file with a non-ASCII filename starting from a UTF-8 name.) Add ufopen() function as a UTF-8 wrapper for fopen(). Change inifile_init() filename argument type to QString.

Leave serial port paths alone ("if your serial port path has non-ANSI characters, you're going to have a bad time").

* Fix valgrind mismatched free/malloc error.

* Correctly encode output paths in native locale on non-Windows

On Mac/Linux, convert output file path from internal UTF-8 to local encoding (which may be UTF-8 or something else) when creating the file. Fixes test_encoding failure.

* Support Unicode paths for .gz files on Windows

Use zlib's Windows-specific gzopen_w() to support Unicode paths.

* Convert UTF-8 to local encoding when opening gzip file on non-Windows

This should fix the test_encoding failure on Travis-CI.

Also, change Windows side to manually allocating wchar_t array; although a bit less elegant, it's very localized and avoids introducing a whole new dependency on std::wstring.

* Use explicit char* to QString conversion in qPrintable

The implicit conversion works, but I prefer the an explicit one (if only as a reminder that we should convert all internal strings to QString).

* Add comment to QString-to-wchar_t[] conversion

... since it may not be immediately obvious what that line of code does.

* Fix merge conflict resolution error

* Fix merge conflict (again)

* Revert shape.cc

* Update gbfile.cc

* Update mkt_logger.cc to use QString tempfile paths.

* Change ufopen() to take QString filename argument.
Where converting from char*, use an explicit QString::fromUtf8() conversion rather than the implicit QString(const char*) constructor.

* Use QFile::exists() and QFile::open() rather than ufopen() to verify a file exists and can be opened.

* Remove redundant call to QFile::exists(). Trivial include cleanup in main.cc.

* Remove superfluous QVector include.

* add newline at EOF.

Merge pull request #124 from tsteven4/appveyor

Appveyor for misc internal builds (debug /opt, 32/64, mscvc/gcc, etc) and start adding features to allow a full release withing Amveor, hopefully in lliest tientethan it takes my VM down download and register the mandatory 4GB of updates!  Hooray kl

Merge pull request #132 from viettaml/getenv-unicode

Support Unicode environment variables.

get gui map preview working with webengine.

Merge pull request #131 from gpsbabel/551

add missing Qt libs for gui on linux.

Support Unicode environment variables.
Create ugetenv() OS-abstraction to return a QString environment variable value. Use it in place of getenv() in most places. Update inifile.cc routines to use QString to support Unicode paths for the gpsbabel.ini file, also greatly streamlining the code.

add missing Qt libs for gui on linux.

The required plugin platforms/libqxcb.so needs
libQt5DBus and libQt5XcbQpa.

Merge pull request #5 from gpsbabel/master

Merge gpsbabel/gpsbabel to viettaml/gpsbabel

Merge pull request #130 from tsteven4/encoding_test

add a test of utf8 filename handling.

add the changes for the latin1 test.

enhance latin1 encoding test.

add a test of utf8 filename handlig.

Merge pull request #129 from viettaml/master

Fix KML writer crash bug in realtime trackign.

script windows release process.

address reviewer concerns wrt inno setup.

Fix KML writer crash bug
When Google Earth uses gpsbabel to import KML in "live" USB mode, kml_wr_position_init() is called before kml_wr_init() resulting in deferencing an uninitialized "writer" pointer. The solution is just to remove the setAutoFormatting(false) call; there's no real need for it as it doesn't measurably impact performance. Also remove other calls to remove nice formatting on timestamps (addressing a FIXME comment), and update the golden test data to match.

Add Q_UNUSED to GDB (moer to come) to hush this class of warnings.

Move is_html (and a fwe in-methods that I could find) from int to bool.

Try to repair damage to main.cc. Sheesh...this is why I should stick to
mechanical changes and merging in older trees of work already done right now...

Bring in more header ordering and modernization. Collapse in work to
simplify (C++-11-isze) some of the pretentious loop iterators in GeoJSON
and some of the silly ones elsewhere. Probably lots more to do.

Merge branch 'cleanup' of https://github.com/gpsbabel/gpsbabel into cleanup

Merge pull request #125 from gpsbabel/cleanup

More strongly order header includes, some const-ification tweaks.

Header sorting, modernization.

Header sorting, modernization.

Header mods.

More header sorting.

Merge branch 'master' into cleanup

Merge pull request #123 from tsteven4/shape

cleanup support for shape files.

Try hoisting windows.h earlier for windows build.

Remove more strays.

Eliminate stray .note files.

Try to make #includes consistently ordered.

update appveyor build.

run basic tests.
build windows installer.
archive installer.

delete obsolete translation file from gui.

mtk test workaround for windows.

Merge pull request #122 from gpsbabel/header_fixes

Remove remaining scraps of electric fence. Favor Valgrind to Efence.

Mechanically update c style foo(void)

cleanup support for shape files.
memory leaks, memory corruption, and other bugs fixed.
on write the required .dbf file is now created.
test added.
document updated.

Remove remaining scraps of electric fence. Valgrind > efence.

Merge pull request #121 from tsteven4/gpxtest

Input redirection can fail with gpx reader on windows

Merge pull request #114 from tsteven4/main_qstrings

Qstring-ification of main.cc.

update comment on QXmlStreamReader failures.

work around QXmlStreamReader issue on windows.

Merge pull request #120 from tsteven4/trackfilter_qsort

ensure track filter merge operation is consistent.

ensure track filter merge operation is consistent.

Merge pull request #119 from tsteven4/gtmtest

make gtm test OS independent.

make gtm test OS independent.

Merge pull request #118 from tsteven4/globalsat2

fix globalsat dump on windows.

fix globalsat dump on windows.

QString-ification of main reviewers suggestions.

Merge pull request #117 from tsteven4/linuxusb

fix GPSBabel.pro for linux adding -lusb.

fix GPSBabel.pro for linux adding -lusb.

Merge pull request #116 from tsteven4/msvc2

build using msvc x64 on AppVeyor.

build using msvc x64 on AppVeyor.

Qstring-ification of main.cc.

Merge pull request #113 from tsteven4/elevation_precision

change default gpx writer elevation precision to 3.

change default gpx writer elevation precision to 3.

Merge pull request #112 from tsteven4/gopal_date

use locale independent string for gopal tracklog.

use locale independent string for gopal tracklog.

More style/warning fixes.

Fix trackfilter.

Fix nmea.

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Reapply warning fixes to skytraq.

Bring skytraq back to original version.

Merge pull request #111 from tsteven4/unicsv_digits

fix unicsv writer msec corruption issue.

fix unicsv writer msec corruption issue.

The final (?) piece for presubmit jooy..